Skip to content

Conversation

aCandidMind
Copy link

@aCandidMind aCandidMind commented Sep 11, 2017

This is a follow-up to #20 which applies to the actually used source branch.

Tested this by checking with middleman build and rackup.

@aCandidMind
Copy link
Author

aCandidMind commented Sep 11, 2017

Note that what's currently online on http://jsonapi-rb.org/guides/serialization/rendering.html with only having the rendering options doesn't seem to match what's in source/. Your new automatic deployment commits on master from some hours ago have put most content in the getting started section. With these changes also reflected in the source branch, this PR here wouldn't make sense, as only the rendering options section would be there. But the rendering page seems a bit out of context like that (the way it's currently online).

@beauby
Copy link
Member

beauby commented Sep 11, 2017

Hi @aCandidMind – thanks for pointing this out! I made the changes and the deploy in a branch and forgot to merge it into source. It should be in sync now!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants